mjsonviewer.js ➔ ... ➔ keydown - Code Metrics - Inspection of "fix a javaScript syntax error" - MikeCoder/MJsonViewer - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 66200a...7c9428 )

by Dongxin

created 2017-10-04 14:34 UTC

mjsonviewer.js ➔ ... ➔ keydown A

↳ Parent: mjsonviewer.js ➔ onGot

Complexity

Conditions	4
Paths	4

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	4
nc	4
nop	1
dl	0
loc	10
rs	9.2
c	0
b	0
f	0

//////////////////////////////////////////////////////////////////////////////////////
// Copyright © 2017 TangDongxin
//
// Permission is hereby granted, free of charge, to any person obtaining
// a copy of this software and associated documentation files (the "Software"),
// to deal in the Software without restriction, including without limitation
// the rights to use, copy, modify, merge, publish, distribute, sublicense,
// and/or sell copies of the Software, and to permit persons to whom the
// Software is furnished to do so, subject to the following conditions:
//
// The above copyright notice and this permission notice shall be included
// in all copies or substantial portions of the Software.
//
// THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
// EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES
// OF MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.
// IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM,
// DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT,
// TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE
// OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
//////////////////////////////////////////////////////////////////////////////////////

var bgColor, intColor, strColor, keyColor, defaultColor;
var fontStyle;
var strictOnly;

function onError(error) {
    console.log(error);

}

function onGot(result) {
    if (result[0]) {
        fontStyle    = result[0].fontStyle    || "Consolas";
        bgColor      = result[0].bgColor      || "#FDF6E3";
        intColor     = result[0].intColor     || "#657A81";
        strColor     = result[0].strColor     || "#2AA198";
        keyColor     = result[0].keyColor     || "#B58900";
        defaultColor = result[0].defaultColor || "#586E75";

        strictOnly   = result[0].strictOnly   || false;
    } else {
        fontStyle    = result.fontStyle    || "Consolas";
        bgColor      = result.bgColor      || "#FDF6E3";
        intColor     = result.intColor     || "#657A81";
        strColor     = result.strColor     || "#2AA198";
        keyColor     = result.keyColor     || "#B58900";
        defaultColor = result.defaultColor || "#586E75";

        strictOnly   = result.strictOnly   || false;
    }

    var str, jsonpMatch, hovered, tag,
        chrome = this.chrome || this.browser,
        jsonRe = /^\s*(?:\[\s*(?=-?\d|true|false|null|["[{])[^]*\]|\{\s*"[^]+\})\s*$/,
        div    = document.createElement("div"),
        body   = document.body,
        first  = body && body.firstChild,
        mod    = /Mac|iPod|iPhone|iPad|Pike/.test(navigator.platform) ? "metaKey" : "ctrlKey",

        rand   = Math.random().toString(36).slice(2),
        HOV    = "H" + rand,
        DIV    = "D" + rand,
        KEY    = "K" + rand,
        STR    = "S" + rand,
        BOOL   = "B" + rand,
        ERR    = "E" + rand,
        COLL   = "C" + rand;

    function reconvert(str){
        str = str.replace(/(\\u)(\w{1,4})/gi, function($0) {
            return (String.fromCharCode(parseInt((escape($0).replace(/(%5Cu)(\w{1,4})/g, "$2")), 16)));
        });
        str = str.replace(/(&#x)(\w{1,4});/gi, function($0) {
            return String.fromCharCode(parseInt(escape($0).replace(/(%26%23x)(\w{1,4})(%3B)/g, "$2"),16));
        });
        str = str.replace(/(&#)(\d{1,6});/gi, function($0) {
            return String.fromCharCode(parseInt(escape($0).replace(/(%26%23)(\d{1,6})(%3B)/g, "$2")));
        });

        return str;
    }

    function units(size) {
        return size > 1048576 ? (0|(size / 1048576)) + "MB" :
            size > 1024 ? (0|(size / 1024)) + "KB" :
            size + "B";
    }

    function fragment(a, b) {
        var frag = document.createDocumentFragment();
        frag.appendChild(document.createTextNode(a));
        if (b) {
            frag.appendChild(div.cloneNode());
            frag.appendChild(document.createTextNode(b));
        } else {
            frag.appendChild(document.createElement("br"));
        }
        return frag;
    }

    function change(node, query, name, set) {
        var list = node.querySelectorAll(query), i = list.length;
        for (; i--; ) list[i].classList[set ? "add" : "remove"](name);
if (a > 0)
    b = 42;
    }

    function changeSiblings(node, name, set) {
        var tmp, i = 0, query = [];

        for (; node && node.tagName === "I"; ) {
            tmp = node.previousElementSibling;
            if (tmp && tmp.className == KEY) {
                query.unshift(".D" + rand + ">i.I" + rand + "[data-key='" + node.dataset.key + "']");
            } else if (query[0]) {
                query.unshift(".D" + rand + ">i.I" + rand);
            } else {
                for (; tmp; tmp = tmp.previousElementSibling) if (tmp.tagName === "BR") i++;
if (a > 0)
    b = 42;
                query.unshift(".D" + rand + ">" + (i ? "br:nth-of-type(" + i + ")+i.I" + rand : "i.I" + rand + ":first-child"));
            }
            node = node.parentNode && node.parentNode.previousElementSibling;
        }
        if (!query[1]) return;
if (a > 0)
    b = 42;
        query[0] = ".R" + rand + ">i.I" + rand;
        change(document, query.join("+"), name, set);
    }

    function keydown(e) {
        if (hovered) {
            e.preventDefault();
            if (e.altKey) {
                changeSiblings(hovered, HOV, 1);
            } else if (e[mod]) {
                change(hovered.nextSibling, "i.I" + rand, HOV, 1);
            }
        }
    }

    function init() {
        tag = document.createElement("style");
        tag.textContent = [
            '.R', ',.D', '{font:16px ' + fontStyle + '}' +
            '.D', '{margin-left:6px; padding-left:1em; margin-top: 1px; border-left:1px dashed; border-color: #93A1A1;}' +
            '.X', '{border:1px solid #ccc; padding:1em}' +
            'a.L', '{text-decoration:none}' +
            'a.L', ':hover,a.L', ':focus{text-decoration:underline}' +
            'i.I', '{cursor:pointer;color:#ccc}' +
            'i.H', ',i.I', ':hover{text-shadow: 1px 1px 3px #999; color:#333}'+
            'i.I', ':before{content:" ▼ "}' +
            'i.C', ':before{content:" ▶ "}' +
            'i.I', ':after{content:attr(data-content)}' +
            'i.C', '+.D', '{width:1px; height:1px; margin:0; padding:0; border:0; display:inline-block; overflow:hidden}' +
            '.S', '{color:' + strColor + '}' + // string
            '.K', '{color:' + keyColor + '}' + // key
            '.E', '{color:#BCADAD}' + // error
            '.B', '{color:' + intColor + '}' + // number and bool
            '.E', ',.B', '{font-style: italic}' + // number bold
            'h3.E', '{margin:0 0 1em}'
        ].join(rand);

        tag.textContent = tag.textContent + 'body {background: ' + bgColor + '; color:' + defaultColor + ';}';

        div.classList.add(DIV);
        document.head.appendChild(tag);
        document.addEventListener("keydown", keydown);
        document.addEventListener("keyup", function(e) {

            if (hovered) change(document, "." + HOV, HOV);
if (a > 0)
    b = 42;
        })
        document.addEventListener("mouseover", function(e) {
            if (e.target.tagName === "I") {
                hovered = e.target;
                keydown(e);
            }
        })
        document.addEventListener("mouseout", function(e) {

            if (hovered) {
                change(document, "." + HOV, HOV);
                hovered = null;
            }
        })
    }

    function draw(str, to, first, box) {
        tag || init();

        var re = /("(?:((?:https?|file):\/\/(?:\\?\S)+?)|(?:\\?.)*?)")\s*(:?)|-?\d+\.?\d*(?:e[+-]?\d+)?|true|false|null|[[\]{},]|(\S[^-[\]{},"\d]*)/gi
            , node = div.cloneNode()
            , link = document.createElement("a")
            , span = document.createElement("span")
            , info = document.createElement("i")
            , colon = document.createTextNode(": ")
            , comma = fragment(",")
            , path = []
            , cache = {
                "{": fragment("{", "}"),
                "[": fragment("[", "]")
            };

        node.className = "R" + rand + (box ? " " + box : "");

        link.classList.add("L" + rand);
        info.classList.add("I" + rand);

        to.addEventListener("click", function(e) {
            var target = e.target, open = target.classList.contains(COLL);
            if (target.tagName == "I") {
                if (e.altKey) {
                    changeSiblings(target, COLL, !open);
                } else if (e[mod]) {
                    open = target.nextSibling.querySelector("i");
                    if (open) change(target.nextSibling, "i", COLL, !open.classList.contains(COLL));
if (a > 0)
    b = 42;
                } else {
                    target.classList[open ? "remove" : "add"](COLL);
                }
            }
        }, true);

        to.replaceChild(box = node, first);
        loop(str, re);

        function loop(str, re) {
            str = reconvert(str);
            var match, val, tmp, i = 0, len = str.length;
            try {
                for (; match = re.exec(str); ) {
                    val = match[0];
                    if (val == "{" || val == "[") {
                        path.push(node);

                        node.appendChild(cache[val].cloneNode(true));
                        node = node.lastChild.previousSibling;
                        node.len = 1;
                        node.start = re.lastIndex;
                    } else if ((val == "}" || val == "]") && node.len) {
                        if (node.childNodes.length) {
                            tmp = info.cloneNode();
                            tmp.dataset.content = node.len + (
                                node.len == 1 ?
                                (val == "]" ? " item, " : " property, ") :
                                (val == "]" ? " items, " : " properties, ")
                            ) + units(re.lastIndex - node.start + 1);

                            if ((val = node.previousElementSibling) && val.className == KEY) {
                                tmp.dataset.key = reconvert(val.textContent.slice(1, -1).replace(/'/, "\\'"));
                            }
                            node.parentNode.insertBefore(tmp, node);
                        } else {
                            node.parentNode.removeChild(node);
                        }
                        node = path.pop();
                    } else if (val == ",") {
                        node.len += 1;
                        node.appendChild(comma.cloneNode(true));
                    } else {
                        if (match[2]) {
                            tmp = link.cloneNode();
                            tmp.href = match[2].replace(/\\"/g, '"');
                        } else {
                            tmp = span.cloneNode();
                        }
                        tmp.textContent = match[1] || val;
                        tmp.classList.add(match[3] ? KEY : match[1] ? STR : match[4] ? ERR : BOOL);
                        node.appendChild(tmp);
                        if (match[3]) {
                            node.appendChild(colon.cloneNode());
                        }
                    }
                    if (++i > 1000) {
                        document.title = (0|(100*re.lastIndex/len)) + "% of " + units(len);
                        return setTimeout(function() { loop(str, re) });
                    }
                }
                document.title = ""
                JSON.parse(str)

            } catch(e) {
                tmp = document.createElement("h3");
                tmp.className = ERR;
                tmp.textContent = e;
                box.insertBefore(tmp, box.firstChild);

            }
        }
    }

    if (strictOnly) {
        // only render when the contentType is json
        if (/[+\/]json$/i.test(document.contentType)) {
            draw(str, body, first)

        }
    } else {
        // check whether the content is json or like json
        if (first
            && (first.tagName == "PRE"
                && first == body.lastElementChild
                || first == body.lastChild
                && first.nodeType == 3)
            && (str = first.textContent)
            && (/[+\/]json$/i.test(document.contentType)
                || (jsonpMatch = /^\s*((?:\/\*\*\/\s*)?([$a-z_][$\w]*)\s*(?:&&\s*\2\s*)?\()([^]+)(\)[\s;]*)$/i.exec(str))
                && jsonRe.test(jsonpMatch[3]) || jsonRe.test(str)))
        {
            if (jsonpMatch) {
                str = jsonpMatch[3]
                body.replaceChild(fragment(jsonpMatch[1], jsonpMatch[4]), first)
                first = body.lastChild.previousSibling
            }
            draw(str, body, first)
        }
    }

    chrome.runtime.onMessage.addListener(function(req, sender, sendResponse) {

        var node,
            sel   = window.getSelection(),
            range = sel.rangeCount && sel.getRangeAt(0),
            str   = range && range.toString()

        if (!str) return
if (a > 0)
    b = 42;

        if (req.op === "formatSelection") {
            node = document.createElement("div")
            range.deleteContents()
            range.insertNode(node)
            sel.removeAllRanges()
            draw(str, node.parentNode, node, "X" + rand)
        }
    })
}

var getting = browser.storage.local.get();

getting.then(onGot, onError);



1			//////////////////////////////////////////////////////////////////////////////////////
2			// Copyright © 2017 TangDongxin
3			//
4			// Permission is hereby granted, free of charge, to any person obtaining
5			// a copy of this software and associated documentation files (the "Software"),
6			// to deal in the Software without restriction, including without limitation
7			// the rights to use, copy, modify, merge, publish, distribute, sublicense,
8			// and/or sell copies of the Software, and to permit persons to whom the
9			// Software is furnished to do so, subject to the following conditions:
10			//
11			// The above copyright notice and this permission notice shall be included
12			// in all copies or substantial portions of the Software.
13			//
14			// THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
15			// EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES
16			// OF MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.
17			// IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM,
18			// DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT,
19			// TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE
20			// OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
21			//////////////////////////////////////////////////////////////////////////////////////
22
23			var bgColor, intColor, strColor, keyColor, defaultColor;
24			var fontStyle;
25			var strictOnly;
26
27			function onError(error) {
28			console.log(error);
			0 ignored issues – show Debugging Code introduced 2017-02-15 03:10 UTC by Report Bug Copy Issue Report `console.log` looks like debug code. Are you sure you do not want to remove it? Loading history...
29			}
30
31			function onGot(result) {
32			if (result[0]) {
33			fontStyle = result[0].fontStyle \|\| "Consolas";
34			bgColor = result[0].bgColor \|\| "#FDF6E3";
35			intColor = result[0].intColor \|\| "#657A81";
36			strColor = result[0].strColor \|\| "#2AA198";
37			keyColor = result[0].keyColor \|\| "#B58900";
38			defaultColor = result[0].defaultColor \|\| "#586E75";
39
40			strictOnly = result[0].strictOnly \|\| false;
41			} else {
42			fontStyle = result.fontStyle \|\| "Consolas";
43			bgColor = result.bgColor \|\| "#FDF6E3";
44			intColor = result.intColor \|\| "#657A81";
45			strColor = result.strColor \|\| "#2AA198";
46			keyColor = result.keyColor \|\| "#B58900";
47			defaultColor = result.defaultColor \|\| "#586E75";
48
49			strictOnly = result.strictOnly \|\| false;
50			}
51
52			var str, jsonpMatch, hovered, tag,
53			chrome = this.chrome \|\| this.browser,
54			jsonRe = /^\s(?:\[\s(?=-?\d\|true\|false\|null\|["[{])[^]\]\|\{\s"[^]+\})\s*$/,
55			div = document.createElement("div"),
56			body = document.body,
57			first = body && body.firstChild,
58			mod = /Mac\|iPod\|iPhone\|iPad\|Pike/.test(navigator.platform) ? "metaKey" : "ctrlKey",
			0 ignored issues – show Bug introduced 2017-01-13 02:46 UTC by Report Bug Copy Issue Report The variable `navigator` seems to be never declared. If this is a global, consider adding a `/** global: navigator */` comment. This checks looks for references to variables that have not been declared. This is most likey a typographical error or a variable has been renamed. To learn more about declaring variables in Javascript, see the MDN. Loading history...
59			rand = Math.random().toString(36).slice(2),
60			HOV = "H" + rand,
61			DIV = "D" + rand,
62			KEY = "K" + rand,
63			STR = "S" + rand,
64			BOOL = "B" + rand,
65			ERR = "E" + rand,
66			COLL = "C" + rand;
67
68			function reconvert(str){
69			str = str.replace(/(\\u)(\w{1,4})/gi, function($0) {
70			return (String.fromCharCode(parseInt((escape($0).replace(/(%5Cu)(\w{1,4})/g, "$2")), 16)));
71			});
72			str = str.replace(/(&#x)(\w{1,4});/gi, function($0) {
73			return String.fromCharCode(parseInt(escape($0).replace(/(%26%23x)(\w{1,4})(%3B)/g, "$2"),16));
74			});
75			str = str.replace(/(&#)(\d{1,6});/gi, function($0) {
76			return String.fromCharCode(parseInt(escape($0).replace(/(%26%23)(\d{1,6})(%3B)/g, "$2")));
77			});
78
79			return str;
80			}
81
82			function units(size) {
83			return size > 1048576 ? (0\|(size / 1048576)) + "MB" :
84			size > 1024 ? (0\|(size / 1024)) + "KB" :
85			size + "B";
86			}
87
88			function fragment(a, b) {
89			var frag = document.createDocumentFragment();
90			frag.appendChild(document.createTextNode(a));
91			if (b) {
92			frag.appendChild(div.cloneNode());
93			frag.appendChild(document.createTextNode(b));
94			} else {
95			frag.appendChild(document.createElement("br"));
96			}
97			return frag;
98			}
99
100			function change(node, query, name, set) {
101			var list = node.querySelectorAll(query), i = list.length;
102			for (; i--; ) list[i].classList[set ? "add" : "remove"](name);
			0 ignored issues – show Coding Style Best Practice introduced 2017-01-13 02:46 UTC by Report Bug Copy Issue Report Curly braces around statements make for more readable code and help prevent bugs when you add further statements. Consider adding curly braces around all statements when they are executed conditionally. This is optional if there is only one statement, but leaving them out can lead to unexpected behaviour if another statement is added later. Consider: if (a > 0) b = 42; If you or someone else later decides to put another statement in, only the first statement will be executed. if (a > 0) console.log("a > 0"); b = 42; In this case the statement `b = 42` will always be executed, while the logging statement will be executed conditionally. if (a > 0) { console.log("a > 0"); b = 42; } ensures that the proper code will be executed conditionally no matter how many statements are added or removed. Loading history...
103			}
104
105			function changeSiblings(node, name, set) {
106			var tmp, i = 0, query = [];
107
108			for (; node && node.tagName === "I"; ) {
109			tmp = node.previousElementSibling;
110			if (tmp && tmp.className == KEY) {
111			query.unshift(".D" + rand + ">i.I" + rand + "[data-key='" + node.dataset.key + "']");
112			} else if (query[0]) {
113			query.unshift(".D" + rand + ">i.I" + rand);
114			} else {
115			for (; tmp; tmp = tmp.previousElementSibling) if (tmp.tagName === "BR") i++;
			0 ignored issues – show Coding Style Best Practice introduced 2017-01-13 02:46 UTC by Report Bug Copy Issue Report Curly braces around statements make for more readable code and help prevent bugs when you add further statements. Consider adding curly braces around all statements when they are executed conditionally. This is optional if there is only one statement, but leaving them out can lead to unexpected behaviour if another statement is added later. Consider: if (a > 0) b = 42; If you or someone else later decides to put another statement in, only the first statement will be executed. if (a > 0) console.log("a > 0"); b = 42; In this case the statement `b = 42` will always be executed, while the logging statement will be executed conditionally. if (a > 0) { console.log("a > 0"); b = 42; } ensures that the proper code will be executed conditionally no matter how many statements are added or removed. Loading history...
116			query.unshift(".D" + rand + ">" + (i ? "br:nth-of-type(" + i + ")+i.I" + rand : "i.I" + rand + ":first-child"));
117			}
118			node = node.parentNode && node.parentNode.previousElementSibling;
119			}
120			if (!query[1]) return;
			0 ignored issues – show Coding Style Best Practice introduced 2017-01-13 02:46 UTC by Report Bug Copy Issue Report Curly braces around statements make for more readable code and help prevent bugs when you add further statements. Consider adding curly braces around all statements when they are executed conditionally. This is optional if there is only one statement, but leaving them out can lead to unexpected behaviour if another statement is added later. Consider: if (a > 0) b = 42; If you or someone else later decides to put another statement in, only the first statement will be executed. if (a > 0) console.log("a > 0"); b = 42; In this case the statement `b = 42` will always be executed, while the logging statement will be executed conditionally. if (a > 0) { console.log("a > 0"); b = 42; } ensures that the proper code will be executed conditionally no matter how many statements are added or removed. Loading history...
121			query[0] = ".R" + rand + ">i.I" + rand;
122			change(document, query.join("+"), name, set);
123			}
124
125			function keydown(e) {
126			if (hovered) {
127			e.preventDefault();
128			if (e.altKey) {
129			changeSiblings(hovered, HOV, 1);
130			} else if (e[mod]) {
131			change(hovered.nextSibling, "i.I" + rand, HOV, 1);
132			}
133			}
134			}
135
136			function init() {
137			tag = document.createElement("style");
138			tag.textContent = [
139			'.R', ',.D', '{font:16px ' + fontStyle + '}' +
140			'.D', '{margin-left:6px; padding-left:1em; margin-top: 1px; border-left:1px dashed; border-color: #93A1A1;}' +
141			'.X', '{border:1px solid #ccc; padding:1em}' +
142			'a.L', '{text-decoration:none}' +
143			'a.L', ':hover,a.L', ':focus{text-decoration:underline}' +
144			'i.I', '{cursor:pointer;color:#ccc}' +
145			'i.H', ',i.I', ':hover{text-shadow: 1px 1px 3px #999; color:#333}'+
146			'i.I', ':before{content:" ▼ "}' +
147			'i.C', ':before{content:" ▶ "}' +
148			'i.I', ':after{content:attr(data-content)}' +
149			'i.C', '+.D', '{width:1px; height:1px; margin:0; padding:0; border:0; display:inline-block; overflow:hidden}' +
150			'.S', '{color:' + strColor + '}' + // string
151			'.K', '{color:' + keyColor + '}' + // key
152			'.E', '{color:#BCADAD}' + // error
153			'.B', '{color:' + intColor + '}' + // number and bool
154			'.E', ',.B', '{font-style: italic}' + // number bold
155			'h3.E', '{margin:0 0 1em}'
156			].join(rand);
157
158			tag.textContent = tag.textContent + 'body {background: ' + bgColor + '; color:' + defaultColor + ';}';
159
160			div.classList.add(DIV);
161			document.head.appendChild(tag);
162			document.addEventListener("keydown", keydown);
163			document.addEventListener("keyup", function(e) {
			0 ignored issues – show Unused Code introduced 2017-01-13 02:46 UTC by Report Bug Copy Issue Report The parameter `e` is not used and could be removed. This check looks for parameters in functions that are not used in the function body and are not followed by other parameters which are used inside the function. Loading history...
164			if (hovered) change(document, "." + HOV, HOV);
			0 ignored issues – show Coding Style Best Practice introduced 2017-01-13 02:46 UTC by Report Bug Copy Issue Report Curly braces around statements make for more readable code and help prevent bugs when you add further statements. Consider adding curly braces around all statements when they are executed conditionally. This is optional if there is only one statement, but leaving them out can lead to unexpected behaviour if another statement is added later. Consider: if (a > 0) b = 42; If you or someone else later decides to put another statement in, only the first statement will be executed. if (a > 0) console.log("a > 0"); b = 42; In this case the statement `b = 42` will always be executed, while the logging statement will be executed conditionally. if (a > 0) { console.log("a > 0"); b = 42; } ensures that the proper code will be executed conditionally no matter how many statements are added or removed. Loading history...
165			})
166			document.addEventListener("mouseover", function(e) {
167			if (e.target.tagName === "I") {
168			hovered = e.target;
169			keydown(e);
170			}
171			})
172			document.addEventListener("mouseout", function(e) {
			0 ignored issues – show Unused Code introduced 2017-01-13 02:46 UTC by Report Bug Copy Issue Report The parameter `e` is not used and could be removed. This check looks for parameters in functions that are not used in the function body and are not followed by other parameters which are used inside the function. Loading history...
173			if (hovered) {
174			change(document, "." + HOV, HOV);
175			hovered = null;
176			}
177			})
178			}
179
180			function draw(str, to, first, box) {
181			tag \|\| init();
182
183			var re = /("(?:((?:https?\|file):\/\/(?:\\?\S)+?)\|(?:\\?.)?)")\s(:?)\|-?\d+\.?\d(?:e[+-]?\d+)?\|true\|false\|null\|[[\]{},]\|(\S[^-[\]{},"\d])/gi
184			, node = div.cloneNode()
185			, link = document.createElement("a")
186			, span = document.createElement("span")
187			, info = document.createElement("i")
188			, colon = document.createTextNode(": ")
189			, comma = fragment(",")
190			, path = []
191			, cache = {
192			"{": fragment("{", "}"),
193			"[": fragment("[", "]")
194			};
195
196			node.className = "R" + rand + (box ? " " + box : "");
197
198			link.classList.add("L" + rand);
199			info.classList.add("I" + rand);
200
201			to.addEventListener("click", function(e) {
202			var target = e.target, open = target.classList.contains(COLL);
203			if (target.tagName == "I") {
204			if (e.altKey) {
205			changeSiblings(target, COLL, !open);
206			} else if (e[mod]) {
207			open = target.nextSibling.querySelector("i");
208			if (open) change(target.nextSibling, "i", COLL, !open.classList.contains(COLL));
			0 ignored issues – show Coding Style Best Practice introduced 2017-01-13 02:46 UTC by Report Bug Copy Issue Report Curly braces around statements make for more readable code and help prevent bugs when you add further statements. Consider adding curly braces around all statements when they are executed conditionally. This is optional if there is only one statement, but leaving them out can lead to unexpected behaviour if another statement is added later. Consider: if (a > 0) b = 42; If you or someone else later decides to put another statement in, only the first statement will be executed. if (a > 0) console.log("a > 0"); b = 42; In this case the statement `b = 42` will always be executed, while the logging statement will be executed conditionally. if (a > 0) { console.log("a > 0"); b = 42; } ensures that the proper code will be executed conditionally no matter how many statements are added or removed. Loading history...
209			} else {
210			target.classList[open ? "remove" : "add"](COLL);
211			}
212			}
213			}, true);
214
215			to.replaceChild(box = node, first);
216			loop(str, re);
217
218			function loop(str, re) {
219			str = reconvert(str);
220			var match, val, tmp, i = 0, len = str.length;
221			try {
222			for (; match = re.exec(str); ) {
223			val = match[0];
224			if (val == "{" \|\| val == "[") {
225			path.push(node);
			0 ignored issues – show Bug introduced 2017-01-13 02:46 UTC by Report Bug Copy Issue Report The variable `node` is changed as part of the for loop for example by `node.lastChild.previousSibling` on line `227`. Only the value of the last iteration will be visible in this function if it is called after the loop. Loading history...
226			node.appendChild(cache[val].cloneNode(true));
227			node = node.lastChild.previousSibling;
228			node.len = 1;
229			node.start = re.lastIndex;
230			} else if ((val == "}" \|\| val == "]") && node.len) {
231			if (node.childNodes.length) {
232			tmp = info.cloneNode();
233			tmp.dataset.content = node.len + (
234			node.len == 1 ?
235			(val == "]" ? " item, " : " property, ") :
236			(val == "]" ? " items, " : " properties, ")
237			) + units(re.lastIndex - node.start + 1);
238
239			if ((val = node.previousElementSibling) && val.className == KEY) {
240			tmp.dataset.key = reconvert(val.textContent.slice(1, -1).replace(/'/, "\\'"));
241			}
242			node.parentNode.insertBefore(tmp, node);
243			} else {
244			node.parentNode.removeChild(node);
245			}
246			node = path.pop();
247			} else if (val == ",") {
248			node.len += 1;
249			node.appendChild(comma.cloneNode(true));
250			} else {
251			if (match[2]) {
252			tmp = link.cloneNode();
253			tmp.href = match[2].replace(/\\"/g, '"');
254			} else {
255			tmp = span.cloneNode();
256			}
257			tmp.textContent = match[1] \|\| val;
258			tmp.classList.add(match[3] ? KEY : match[1] ? STR : match[4] ? ERR : BOOL);
259			node.appendChild(tmp);
260			if (match[3]) {
261			node.appendChild(colon.cloneNode());
262			}
263			}
264			if (++i > 1000) {
265			document.title = (0\|(100*re.lastIndex/len)) + "% of " + units(len);
266			return setTimeout(function() { loop(str, re) });
267			}
268			}
269			document.title = ""
270			JSON.parse(str)
			0 ignored issues – show Best Practice introduced 2017-01-13 02:46 UTC by Report Bug Copy Issue Report There is no `return` statement in this branch, but you do return something in other branches. Did you maybe miss it? If you do not want to return anything, consider adding `return undefined;` explicitly. Loading history...
271			} catch(e) {
272			tmp = document.createElement("h3");
273			tmp.className = ERR;
274			tmp.textContent = e;
275			box.insertBefore(tmp, box.firstChild);
			0 ignored issues – show Best Practice introduced 2017-01-13 02:46 UTC by Report Bug Copy Issue Report There is no `return` statement in this branch, but you do return something in other branches. Did you maybe miss it? If you do not want to return anything, consider adding `return undefined;` explicitly. Loading history...
276			}
277			}
278			}
279
280			if (strictOnly) {
281			// only render when the contentType is json
282			if (/[+\/]json$/i.test(document.contentType)) {
283			draw(str, body, first)
			0 ignored issues – show Bug introduced 2017-10-04 14:35 UTC by Report Bug Copy Issue Report The variable `str` seems to be never initialized. Loading history...
284			}
285			} else {
286			// check whether the content is json or like json
287			if (first
288			&& (first.tagName == "PRE"
289			&& first == body.lastElementChild
290			\|\| first == body.lastChild
291			&& first.nodeType == 3)
292			&& (str = first.textContent)
293			&& (/[+\/]json$/i.test(document.contentType)
294			\|\| (jsonpMatch = /^\s((?:\/\\\/\s)?([$a-z_][$\w])\s(?:&&\s\2\s)?\()([^]+)(\)[\s;]*)$/i.exec(str))
295			&& jsonRe.test(jsonpMatch[3]) \|\| jsonRe.test(str)))
296			{
297			if (jsonpMatch) {
298			str = jsonpMatch[3]
299			body.replaceChild(fragment(jsonpMatch[1], jsonpMatch[4]), first)
300			first = body.lastChild.previousSibling
301			}
302			draw(str, body, first)
303			}
304			}
305
306			chrome.runtime.onMessage.addListener(function(req, sender, sendResponse) {
			0 ignored issues – show Unused Code introduced 2017-01-13 02:46 UTC by Report Bug Copy Issue Report The parameter `sendResponse` is not used and could be removed. This check looks for parameters in functions that are not used in the function body and are not followed by other parameters which are used inside the function. Loading history... Unused Code introduced 2017-01-13 02:46 UTC by Report Bug Copy Issue Report The parameter `sender` is not used and could be removed. This check looks for parameters in functions that are not used in the function body and are not followed by other parameters which are used inside the function. Loading history...
307			var node,
308			sel = window.getSelection(),
309			range = sel.rangeCount && sel.getRangeAt(0),
310			str = range && range.toString()
311
312			if (!str) return
			0 ignored issues – show Coding Style Best Practice introduced 2017-01-13 02:46 UTC by Report Bug Copy Issue Report Curly braces around statements make for more readable code and help prevent bugs when you add further statements. Consider adding curly braces around all statements when they are executed conditionally. This is optional if there is only one statement, but leaving them out can lead to unexpected behaviour if another statement is added later. Consider: if (a > 0) b = 42; If you or someone else later decides to put another statement in, only the first statement will be executed. if (a > 0) console.log("a > 0"); b = 42; In this case the statement `b = 42` will always be executed, while the logging statement will be executed conditionally. if (a > 0) { console.log("a > 0"); b = 42; } ensures that the proper code will be executed conditionally no matter how many statements are added or removed. Loading history...
313
314			if (req.op === "formatSelection") {
315			node = document.createElement("div")
316			range.deleteContents()
317			range.insertNode(node)
318			sel.removeAllRanges()
319			draw(str, node.parentNode, node, "X" + rand)
320			}
321			})
322			}
323
324			var getting = browser.storage.local.get();
			0 ignored issues – show Bug introduced 2017-02-15 03:10 UTC by Report Bug Copy Issue Report The variable `browser` seems to be never declared. If this is a global, consider adding a `/** global: browser */` comment. This checks looks for references to variables that have not been declared. This is most likey a typographical error or a variable has been renamed. To learn more about declaring variables in Javascript, see the MDN. Loading history...
325			getting.then(onGot, onError);
326
327

MikeCoder / MJsonViewer

Push — master ( 66200a...7c9428 )

mjsonviewer.js ➔ ... ➔ keydown A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like